Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[TECH] Migration de la route /api/admin/sessions/publish-in-batch (PIX-15768). #10842

Merged
merged 1 commit into from
Dec 18, 2024
Merged

[TECH] Migration de la route /api/admin/sessions/publish-in-batch (PIX-15768). #10842

merged 1 commit into from
Dec 18, 2024

Conversation

alexandrecoin
Copy link
Contributor

@alexandrecoin alexandrecoin commented Dec 17, 2024 •
edited
Loading

🎄 Problème

La route /api/admin/sessions/publish-in-batch n’est pas migrée.

🎁 Proposition

On la migre

🧦 Remarques

🎅 Pour tester

  • Se connecter à pix-certif avec [email protected]
  • Créer au minimum deux sessions avec un candidat par session (ajouter votre adresse email pour l'envoi des résultats)
  • Commencer chacune des sessions (ex: à l'aide du compte[email protected] sur mon-pix)
  • Finaliser les sessions sur pix-certif
  • Se connecter à pix-admin avec [email protected]
  • Sur la page des sessions, aller sur l'onglet sessions V3 à publier
  • Publier en masse les deux sessions fraîchement finalisées et vérifier que tout se passe bien (les sessions sont publiées et vous recevez les mails de résultats)

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@alexandrecoin alexandrecoin force-pushed the pix-15768-move-publish-in-batch-route-to-src branch from a54e045 to f967542 Compare December 17, 2024 17:02
@alexandrecoin alexandrecoin self-assigned this Dec 17, 2024
@alexandrecoin alexandrecoin marked this pull request as ready for review December 17, 2024 17:13
@alexandrecoin alexandrecoin requested a review from a team as a code owner December 17, 2024 17:13
@alexandrecoin alexandrecoin changed the title refactor(api): move publish in batch route to src [TECH] Migration de la route /api/admin/sessions/publish-in-batch (PIX-15768). Dec 17, 2024
@yaf
Copy link
Member

yaf commented Dec 18, 2024

✔️ test fonctionnel OK !

@yaf yaf added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Dec 18, 2024
yaf
yaf approved these changes Dec 18, 2024
View reviewed changes
Copy link
Member

@yaf yaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 :shipit:

@pix-service-auto-merge pix-service-auto-merge merged commit 8a5435c into dev Dec 18, 2024
22 of 23 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15768-move-publish-in-batch-route-to-src branch December 18, 2024 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yaf yaf yaf approved these changes

Assignees

@alexandrecoin alexandrecoin

Labels
Func Review OK PO validated functionally the PR 🚀 Ready to Merge team-certif Tech Review OK
Projects
None yet
Milestone
Bounded context migration
Development

Successfully merging this pull request may close these issues.
4 participants
@alexandrecoin @pix-bot-github @yaf @pix-service-auto-merge